Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Options page - choose popup arrow location #244

Merged

Conversation

pietromartino
Copy link
Contributor

@pietromartino pietromartino commented Dec 11, 2023

Describe your changes

Options Page allowing users to configure autonomously most of the options described in the How to page.
Additionally, the popup arrow can be moved around the corners of the screen either orizontally and vertically.

All options:

  • Popup button orientation (Horizontal/Vertical and relative position inside the page)
  • Flow Scrollability
  • Inspect page - Show table borders
  • Always open links in a new tab
  • Open Permission Set / Permission Set Group summary from shortcuts
  • Search metadata from Shortcut tab
  • Disable query input autofocus
  • API Version
  • Consumer Key
  • CSV Separator in Data Export

Preview

image

image

image

image

Ability to move popup icon is a feature request #145 by @Tratius

@tprouvot and fellow contributors, let me know if you have any suggestion on how to organize all options into different tabs, it's quite easy to do :)

Issue ticket number and link

Checklist before requesting a review

  • I have read and understand the Contributions section
  • Target branch is releaseCandidate and not master
  • I have performed a self-review of my code
  • I ran the unit tests and my PR does not break any tests
  • I documented the changes I've made on the CHANGES.md and followed actual conventions
  • [] I added a new section on how-to.md (optional)

@pietromartino pietromartino marked this pull request as draft December 12, 2023 09:12
@pietromartino
Copy link
Contributor Author

Found a bug in popup open/close behaviour - reverting to draft

@tprouvot
Copy link
Owner

Hi @pietromartino thanks for your work ! 👏
Really like the fact that you integrate the background image like on the other pages to unify the UI 👍

@pietromartino pietromartino marked this pull request as ready for review December 14, 2023 10:44
@tprouvot
Copy link
Owner

Hi @pietromartino thanks for the PR !
As discussed earlier, the addition of the slds.css and salesforce-lightning-design-system.css files increase extension size.
Is there lots of classes that are used in the setting page ?

@tprouvot tprouvot merged commit 91eca90 into tprouvot:releaseCandidate Jan 2, 2024
AntoineLeleu-Salesforce pushed a commit to AntoineLeleu-Salesforce/Salesforce-Inspector-reloaded that referenced this pull request Jan 4, 2024
## Describe your changes

Options Page allowing users to configure autonomously most of the
options described in the [How to
page](https://github.com/tprouvot/Salesforce-Inspector-reloaded/blob/master/docs/how-to.md).
Additionally, the popup arrow can be moved around the corners of the
screen either orizontally and vertically.

All options:

- **Popup button orientation (Horizontal/Vertical and relative position
inside the page)**
- Flow Scrollability
- Inspect page - Show table borders
- Always open links in a new tab
- Open Permission Set / Permission Set Group summary from shortcuts
- Search metadata from Shortcut tab
- Disable query input autofocus
- API Version
- Consumer Key
- CSV Separator in Data Export

### Preview 


![image](https://github.com/tprouvot/Salesforce-Inspector-reloaded/assets/17877327/7b54a9d9-60f1-4c65-99ad-5abf6518db36)


![image](https://github.com/tprouvot/Salesforce-Inspector-reloaded/assets/17877327/2ecafc2b-0d81-4cc3-b186-91304f8ac570)


![image](https://github.com/tprouvot/Salesforce-Inspector-reloaded/assets/17877327/3fd60297-9db0-4777-b77a-4d6379874332)


![image](https://github.com/tprouvot/Salesforce-Inspector-reloaded/assets/17877327/60960f82-46a1-499a-b286-d7ada0c7e323)

Ability to move popup icon is a feature request tprouvot#145 by @Tratius

@tprouvot and fellow contributors, let me know if you have any
suggestion on how to organize all options into different tabs, it's
quite easy to do :)

## Issue ticket number and link

## Checklist before requesting a review
- [x] I have read and understand the [Contributions
section](https://github.com/tprouvot/Salesforce-Inspector-reloaded#contributions)
- [x] Target branch is releaseCandidate and not master
- [x] I have performed a self-review of my code
- [x] I ran the [unit
tests](https://github.com/tprouvot/Salesforce-Inspector-reloaded#unit-tests)
and my PR does not break any tests
- [x] I documented the changes I've made on the
[CHANGES.md](https://github.com/tprouvot/Salesforce-Inspector-reloaded/blob/master/CHANGES.md)
and followed actual conventions
- [] I added a new section on
[how-to.md](https://github.com/tprouvot/Salesforce-Inspector-reloaded/blob/master/docs/how-to.md)
(optional)

---------

Co-authored-by: Pietro Martino <[email protected]>
Co-authored-by: Thomas Prouvot <[email protected]>
AntoineLeleu-Salesforce pushed a commit to AntoineLeleu-Salesforce/Salesforce-Inspector-reloaded that referenced this pull request Jan 4, 2024
## Describe your changes

Options Page allowing users to configure autonomously most of the
options described in the [How to
page](https://github.com/tprouvot/Salesforce-Inspector-reloaded/blob/master/docs/how-to.md).
Additionally, the popup arrow can be moved around the corners of the
screen either orizontally and vertically.

All options:

- **Popup button orientation (Horizontal/Vertical and relative position
inside the page)**
- Flow Scrollability
- Inspect page - Show table borders
- Always open links in a new tab
- Open Permission Set / Permission Set Group summary from shortcuts
- Search metadata from Shortcut tab
- Disable query input autofocus
- API Version
- Consumer Key
- CSV Separator in Data Export

### Preview 


![image](https://github.com/tprouvot/Salesforce-Inspector-reloaded/assets/17877327/7b54a9d9-60f1-4c65-99ad-5abf6518db36)


![image](https://github.com/tprouvot/Salesforce-Inspector-reloaded/assets/17877327/2ecafc2b-0d81-4cc3-b186-91304f8ac570)


![image](https://github.com/tprouvot/Salesforce-Inspector-reloaded/assets/17877327/3fd60297-9db0-4777-b77a-4d6379874332)


![image](https://github.com/tprouvot/Salesforce-Inspector-reloaded/assets/17877327/60960f82-46a1-499a-b286-d7ada0c7e323)

Ability to move popup icon is a feature request tprouvot#145 by @Tratius

@tprouvot and fellow contributors, let me know if you have any
suggestion on how to organize all options into different tabs, it's
quite easy to do :)

## Issue ticket number and link

## Checklist before requesting a review
- [x] I have read and understand the [Contributions
section](https://github.com/tprouvot/Salesforce-Inspector-reloaded#contributions)
- [x] Target branch is releaseCandidate and not master
- [x] I have performed a self-review of my code
- [x] I ran the [unit
tests](https://github.com/tprouvot/Salesforce-Inspector-reloaded#unit-tests)
and my PR does not break any tests
- [x] I documented the changes I've made on the
[CHANGES.md](https://github.com/tprouvot/Salesforce-Inspector-reloaded/blob/master/CHANGES.md)
and followed actual conventions
- [] I added a new section on
[how-to.md](https://github.com/tprouvot/Salesforce-Inspector-reloaded/blob/master/docs/how-to.md)
(optional)

---------

Co-authored-by: Pietro Martino <[email protected]>
Co-authored-by: Thomas Prouvot <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants